src/interface.ts (1)

61-61: 建议添加 JSDoc 注释说明 keyboard 属性的用途。

新增的 keyboard 属性缺少文档注释，建议添加说明以提高 API 的可读性和可维护性。例如：

   open?: boolean;
+  /** Whether to enable keyboard interaction (Esc to close, Left/Right to navigate). Default: true */
   keyboard?: boolean;
   defaultOpen?: boolean;

src/Tour.tsx (1)

195-201: 考虑使用 useEffect 代替 useLayoutEffect。

事件监听器的注册和清理逻辑正确，依赖项数组也准确。但是，对于键盘事件监听器的注册，使用 useEffect 而非 useLayoutEffect 更合适，因为这个操作不需要在浏览器绘制之前同步执行，使用 useEffect 可以避免阻塞视觉更新。

建议修改：

- useLayoutEffect(() => {
+ useEffect(() => {
    if (!mergedOpen) return;
    window.addEventListener('keydown', keyboardHandler);
    return () => {
      window.removeEventListener('keydown', keyboardHandler);
    };
  }, [mergedOpen, keyboardHandler]);

注意：需要在文件顶部添加 useEffect 的导入：

- import { useMemo } from 'react';
+ import { useMemo, useEffect } from 'react';

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

src/Tour.tsx (1)

168-204: 建议采用现代 API 并改进代码结构。

当前实现已经解决了之前评审中关于 preventDefault 调用时机和输入元素过滤的问题，这很好！但仍有改进空间：

1. 使用 e.key 替代 e.keyCode：keyCode 已被弃用，应使用 e.key 以符合现代 Web 标准。
2. 使用 switch 语句：相比多个 if 语句，switch 语句可读性和可维护性更好。

应用以下差异来改进实现：

  const keyboardHandler = useEvent((e: KeyboardEvent) => {
    // Ignore keyboard events from input-like elements to avoid interfering when typing
    const el = e.target as HTMLElement | null;
    if (
        el?.tagName === 'INPUT' ||
        el?.tagName === 'TEXTAREA' ||
        el?.tagName === 'SELECT' ||
        el?.isContentEditable
      ) {
        return;
      }

-   if (keyboard && e.keyCode === KeyCode.ESC) {
-     if (mergedClosable !== null) {
-       e.stopPropagation();
-       e.preventDefault();
-       handleClose();
-     }
-     return;
-   }
-
-   if (keyboard && e.keyCode === KeyCode.LEFT) {
-     if (mergedCurrent > 0) {
-       e.preventDefault();
-       onInternalChange(mergedCurrent - 1);
-     }
-     return;
-   }
-
-   if (keyboard && e.keyCode === KeyCode.RIGHT) {
-     if (mergedCurrent < steps.length - 1) {
-       e.preventDefault();
-       onInternalChange(mergedCurrent + 1);
-     }
-     return;
-   }
+   if (!keyboard) {
+     return;
+   }
+
+   switch (e.key) {
+     case 'Escape':
+       if (mergedClosable !== null) {
+         e.stopPropagation();
+         e.preventDefault();
+         handleClose();
+       }
+       break;
+     case 'ArrowLeft':
+       if (mergedCurrent > 0) {
+         e.preventDefault();
+         onInternalChange(mergedCurrent - 1);
+       }
+       break;
+     case 'ArrowRight':
+       if (mergedCurrent < steps.length - 1) {
+         e.preventDefault();
+         onInternalChange(mergedCurrent + 1);
+       }
+       break;
+     default:
+       break;
+   }
  });

src/Tour.tsx (1)

7-8: 建议使用现代的 e.key API 替代已弃用的 KeyCode。

KeyCode 已被弃用，建议使用 e.key 属性来检测按键。这样可以移除对 KeyCode 的导入依赖，并使代码更符合现代 Web 标准。

应用以下差异以移除 KeyCode 导入：

 import useLayoutEffect from '@rc-component/util/lib/hooks/useLayoutEffect';
 import useEvent from '@rc-component/util/lib/hooks/useEvent';
-import KeyCode from '@rc-component/util/lib/KeyCode';
 import useControlledState from '@rc-component/util/lib/hooks/useControlledState';

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro